[No QA] [Payment due @truph01] Docs updates for E/A#96947 add Download receipts option to reports - #97740
[No QA] [Payment due @truph01] Docs updates for E/A#96947 add Download receipts option to reports#97740MelvinBot wants to merge 2 commits into
Conversation
Co-authored-by: truph01 <truph01@users.noreply.github.com>
|
@MelvinBot Mark this PR as ready |
|
✅ Marked this PR as ready for review. This is a docs-only change (a single HelpDot markdown file), so there's nothing to test in the app — no app-side verification was run. |
HelpDot Documentation ReviewOverall Assessment Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: Detailed line-by-line feedback has been provided as inline comments. |
|
🎯 @truph01, thanks for reviewing and testing this PR! 🎉 A payment issue will be created for your review once this PR is deployed to production. If payment is not needed (e.g., regression PR review fix etc), react with 👎 to this comment to prevent the payment issue from being created. |
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? |
|
@strepanier03 Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
|
A preview of your ExpensifyHelp changes have been deployed to https://f0eda561.helpdot.pages.dev ⚡️ Updated articles: |
|
The only failing check is This isn't a code failure, so there's nothing to fix or push. This check gates on the PR having an independent human approval, and it stays red until a reviewer (someone other than the author) approves the PR. It's the same reason To resolve: get an independent reviewer to approve this PR — no code change is needed. |
|
Still the same as my previous reply — nothing has changed and there's no code fix to push. The only red check is To resolve: get an independent reviewer to approve this PR. No commit or code change from me will clear it. |
Explanation of Change
#96947 adds a Download receipts option to the More menu on an expense report. It downloads all of the receipts attached to that report as a single zip file, with a status window that prepares the file and downloads it automatically (with a Download file fallback button).
This updates the help site to document the new option.
What changed
How-to-Export-Reports.md— Added a new section, How to download the receipts on a report, alongside the existing single-report Export (CSV) and Download as PDF flows, since Download receipts lives in the same report More menu.description,keywords, andinternalScopefrontmatter to include receipts.Notes / verification
Download receipts(common.downloadReceipts).reportTransactions.some(hasReceipt)inReportSecondaryActionUtils.ts, so it only appears when a report has at least one receipt.exportReceiptsToZipcalls theExportReceiptsToZipcommand and produces a zip download; the status modal handles preparing → ready (auto-download) → partial/no-receipts/failed states.Fixed Issues
Related to #96947
@truph01, please review the files changed and confirm they reflect the current behavior. Then post a new comment saying
@MelvinBot mark this PR as ready for review.